Skip to content

Show license preview page for admins and plugin owners#355

Merged
simonhamp merged 1 commit intomainfrom
show-license-preview-page
Apr 13, 2026
Merged

Show license preview page for admins and plugin owners#355
simonhamp merged 1 commit intomainfrom
show-license-preview-page

Conversation

@simonhamp
Copy link
Copy Markdown
Member

Summary

  • Allow admins and plugin authors to view the license page for paid plugins before they are approved, matching the existing preview behavior on the plugin show page
  • Add the same amber "Preview" banner to the license page explaining why the user can see it and why others can't (plugin status or de-listed state)
  • Add admin/owner bypass for approval, active, and accessible-price checks in the license controller

Test plan

  • Guest cannot view pending paid plugin license page (404)
  • Regular user cannot view pending paid plugin license page (404)
  • Admin can view pending paid plugin license page (200)
  • Owner can view pending paid plugin license page (200)
  • Admin sees preview banner with status on pending plugin
  • Owner sees preview banner with status on pending plugin
  • Approved active plugin shows no preview banner
  • Admin sees "de-listed" banner on inactive approved plugin
  • Guest cannot view de-listed plugin license (404)
  • Free plugin license still returns 404
  • Plugin without license_html still returns 404

🤖 Generated with Claude Code

Allow admins and plugin authors to view the license page for paid
plugins before they are approved, matching the existing preview
behavior on the plugin show page. Adds an amber preview banner
explaining the plugin's visibility status.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@simonhamp simonhamp marked this pull request as ready for review April 13, 2026 00:39
@simonhamp simonhamp merged commit 07f1e42 into main Apr 13, 2026
2 checks passed
@simonhamp simonhamp deleted the show-license-preview-page branch April 13, 2026 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant